Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[JSBSim] Upgrade to v1.1.12 #5572

Merged
merged 13 commits into from
Sep 25, 2022
Merged

[JSBSim] Upgrade to v1.1.12 #5572

merged 13 commits into from
Sep 25, 2022

Conversation

bcoconni
Copy link
Contributor

As the title says: the purpose of this PR is to upgrade the JSBSim package to the last version of the upstream library.

@bcoconni
Copy link
Contributor Author

The build is crashing with a cryptic error message which seems to be somehow linked to the dependency package libjulia_jll. Can someone provide me with some help ?

image

@giordano
Copy link
Member

If you search for that UUID (a83860b7-747b-57cf-bf1f-3e79990d037f) in this repository, you'll find lots of answer: https://github.com/JuliaPackaging/Yggdrasil/search?q=a83860b7-747b-57cf-bf1f-3e79990d037f&type=code 🙂

@giordano
Copy link
Member

I believe you may want to follow what other packages do here, see for example https://github.com/JuliaPackaging/Yggdrasil/blob/e86c5d9c3d34f70e4451ba24644051ddd52cb9dd/L/libsingular_julia/build_tarballs.jl. Otherwise this going to work only with Julia 1.6.

@bcoconni
Copy link
Contributor Author

Phew ! That was not a smooth ride 😄

As far as I can tell the PR is ready to be merged (I hope you're using "squash & merge" because there is little point in keeping all the hoops I had to jump through 😞)

@giordano giordano merged commit afbeb78 into JuliaPackaging:master Sep 25, 2022
@bcoconni
Copy link
Contributor Author

Great 👍 ! Thanks for your help @giordano 😄

@bcoconni bcoconni deleted the JSBSim branch September 25, 2022 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants